Bck 44/del role in resp reg user #86
Conversation
app/core/user.py
Outdated
| @@ -67,7 +67,7 @@ async def on_after_register( | |||
| 'Welcome to QActf.', | |||
| f'{URL}/auth/confirm/{user_uid}/{user_code}', | |||
| ) | |||
| await cofirm_email.send_email_message() | |||
| # await cofirm_email.send_email_message() | |||
There was a problem hiding this comment.
Закомментированный код необходимо удалять :)
There was a problem hiding this comment.
принял. действительно)
app/requirements.txt
Outdated
| @@ -13,7 +13,7 @@ asyncpg==0.29.0 | |||
| attrs==23.1.0 | |||
| Babel==2.14.0 | |||
| backcall==0.2.0 | |||
| bcrypt==4.1.1 | |||
| bcrypt==4.0.1 | |||
There was a problem hiding this comment.
Почему мы здесь понижаем версию библиотеки?
There was a problem hiding this comment.
При сборке контейнеров возникала ошибка: AttributeError: module 'bcrypt' has no attribute 'about'
Не понял влияло ли это на работоспособность, просто мозолила глаза. Наткнулся на обсуждение подобной проблемы.
app/requirements.txt
Outdated
| @@ -13,7 +13,7 @@ asyncpg==0.29.0 | |||
| attrs==23.1.0 | |||
| Babel==2.14.0 | |||
| backcall==0.2.0 | |||
| bcrypt==4.1.1 | |||
| bcrypt==4.0.1 | |||
There was a problem hiding this comment.
При сборке контейнеров возникала ошибка: AttributeError: module 'bcrypt' has no attribute 'about'
Не понял влияло ли это на работоспособность, просто мозолила глаза. Наткнулся на обсуждение подобной проблемы.
app/core/user.py
Outdated
| @@ -67,7 +67,7 @@ async def on_after_register( | |||
| 'Welcome to QActf.', | |||
| f'{URL}/auth/confirm/{user_uid}/{user_code}', | |||
| ) | |||
| await cofirm_email.send_email_message() | |||
| # await cofirm_email.send_email_message() | |||
There was a problem hiding this comment.
принял. действительно)
Уверен, что решение задачи ВСК-44 должно быть изящнее. Но пока получилось только так, с учетом того, насколько успел разобраться со структурой незнакомого мне FastAPI,
отправку подтверждающего email пока закоментил: не получилось пока победить этот функционал.